Skip to content

Remove Psalm#131

Merged
HypeMC merged 1 commit intosofascore:1.xfrom
HypeMC:remove-psalm
Dec 17, 2025
Merged

Remove Psalm#131
HypeMC merged 1 commit intosofascore:1.xfrom
HypeMC:remove-psalm

Conversation

@HypeMC
Copy link
Copy Markdown
Member

@HypeMC HypeMC commented Dec 17, 2025

I don't think there's a point in having two tools that do the same thing. After testing both Psalm and PHPStan these last few years, I'm convinced PHPStan is a better fit for us, so let's remove Psalm.

@HypeMC HypeMC requested a review from Brajk19 December 17, 2025 04:52
@HypeMC HypeMC self-assigned this Dec 17, 2025
@HypeMC HypeMC requested a review from plantas December 17, 2025 04:52
@codecov
Copy link
Copy Markdown

codecov Bot commented Dec 17, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.23%. Comparing base (bb26936) to head (99afaa9).
⚠️ Report is 1 commits behind head on 1.x.

Additional details and impacted files
@@            Coverage Diff            @@
##                1.x     #131   +/-   ##
=========================================
  Coverage     97.23%   97.23%           
  Complexity      491      491           
=========================================
  Files            62       62           
  Lines          1373     1373           
=========================================
  Hits           1335     1335           
  Misses           38       38           
Flag Coverage Δ
8.1 97.23% <ø> (ø)
8.2 97.15% <ø> (+0.72%) ⬆️
8.3 96.93% <ø> (+0.29%) ⬆️
8.4 96.93% <ø> (+0.51%) ⬆️
8.5 96.93% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@HypeMC HypeMC merged commit 13ec0bc into sofascore:1.x Dec 17, 2025
37 checks passed
@HypeMC HypeMC deleted the remove-psalm branch December 17, 2025 08:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants